Skip to content

feat: support vanna tools#501

Closed
BhAem wants to merge 8 commits intovolcengine:mainfrom
BhAem:feat/vanna
Closed

feat: support vanna tools#501
BhAem wants to merge 8 commits intovolcengine:mainfrom
BhAem:feat/vanna

Conversation

@BhAem
Copy link
Collaborator

@BhAem BhAem commented Feb 9, 2026

No description provided.

@BhAem BhAem marked this pull request as draft February 9, 2026 08:12
BhAem and others added 5 commits February 25, 2026 09:00
…hain (volcengine#528)

* During the conversion process from langchain to veadk, correct the read and write operation logic of long-term memory

- Modify save_session.py to:
1. Filter out messages with potentially empty content in AIMessage
2. Correctly handle ToolMessage, ignoring it regardless of its content
3. Only save HumanMessage and AIMessage messages that have actual content

- In viking_memory.py:
1. Modify the batch() method:
   - Add the corresponding result to the results list for each operation
   - PutOp: Add None
   - GetOp: Correctly add the result of self._apply_get_op(op)
   - SearchOp: Correctly add the result of self._apply_search_op(op)
2. Modify the _apply_search_op() method:
   - Use op.limit or 1 to correctly handle search limits

* release new version to 0.5.25

---------

Co-authored-by: yaozheng-fang <fangyaozheng@bytedance.com>
@BhAem BhAem closed this Mar 3, 2026
@BhAem BhAem deleted the feat/vanna branch March 3, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants